Skip to content

Conversation

@yehorkardash
Copy link
Contributor

Summary

This PR enables proxy support for MCP Client Tool. It updates fetch method in MCPClient constructor with a custom fetch that uses ProxyAgent

Testing

  1. Set up local proxy (e.g., tinyproxy)
  2. Test with various combinations of HTTP_PROXY, HTTPS_PROXY, ALL_PROXY, NO_PROXY
  3. Verify MCP client tool makes requests through proxy
  4. Confirm NO_PROXY patterns work correctly

Related Linear tickets, Github issues, and Community forum posts

https://linear.app/n8n/issue/NODE-3754/mcp-client-tool-proxy-support

Review / Merge checklist

  • PR title and summary are descriptive. (conventions)
  • Docs updated or follow-up ticket created.
  • Tests included.
  • PR Labeled with release/backport (if the PR is an urgent fix that needs to be backported)

@n8n-assistant n8n-assistant bot added core Enhancement outside /nodes-base and /editor-ui n8n team Authored by the n8n team labels Nov 11, 2025
@codecov
Copy link

codecov bot commented Nov 11, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 4 files

Copy link
Contributor

@RomanDavydchuk RomanDavydchuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works as expected

@currents-bot
Copy link

currents-bot bot commented Nov 17, 2025

E2E Tests: n8n tests passed after 8m 9.8s

🟢 585 · 🔴 0 · ⚪️ 12 · 🟣 2

View Run Details

Run Details

  • Project: n8n

  • Groups: 2

  • Framework: Playwright

  • Run Status: Passed

  • Commit: a0d65f5

  • Spec files: 96

  • Overall tests: 597

  • Duration: 8m 9.8s

  • Parallelization: 8

Groups

GroupId Results Spec Files Progress
ui 🟢 530 · 🔴 0 · ⚪️ 12 · 🟣 1 89 / 89
ui:isolated 🟢 55 · 🔴 0 · ⚪️ 0 · 🟣 1 7 / 7


This message was posted automatically by currents.dev | Integration Settings

@RomanDavydchuk RomanDavydchuk merged commit d9e2dc2 into master Nov 17, 2025
45 checks passed
@RomanDavydchuk RomanDavydchuk deleted the node-3754-mcp-client-tool-proxy-support branch November 17, 2025 09:04
@n8n-assistant
Copy link

n8n-assistant bot commented Nov 18, 2025

Got released with [email protected]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Enhancement outside /nodes-base and /editor-ui n8n team Authored by the n8n team Released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants